Skip to content

Update backup registry secret name#3054

Open
dkwon17 wants to merge 4 commits intoeclipse-che:mainfrom
dkwon17:registry-secret
Open

Update backup registry secret name#3054
dkwon17 wants to merge 4 commits intoeclipse-che:mainfrom
dkwon17:registry-secret

Conversation

@dkwon17
Copy link
Copy Markdown
Contributor

@dkwon17 dkwon17 commented Mar 26, 2026

What does this pull request change?

A bug was discovered when trying to restore from a private quay.io repo, where the pull secret was not accessible to the restore init container.

A workaround is to have the pull secret be named devworkspace-backup-registry-auth.

What issues does this pull request fix or reference?

Specify the version of the product this pull request applies to

DS 3.27.

Pull Request checklist

The author and the reviewers validate the content of this pull request with the following checklist, in addition to the automated tests.

  • Any procedure:
    • Successfully tested.
  • Any page or link rename:
    • The page contains a redirection for the previous URL.
    • Propagate the URL change in:
  • Builds on Eclipse Che hosted by Red Hat.
  • the Validate language on files added or modified step reports no vale warnings.

Signed-off-by: dkwon17 <dakwon@redhat.com>
@dkwon17 dkwon17 requested a review from ibuziuk March 26, 2026 19:53
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 26, 2026

🎊 Navigate the preview: https://69c6fd2fcdc7b5da88032b8c--eclipse-che-docs-pr.netlify.app 🎊

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 26, 2026

Click here to review and test in web IDE: Contribute

Copy link
Copy Markdown
Contributor

@gtrivedi88 gtrivedi88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some suggestions, otherwise looks good.

enable: true
registry:
authSecret: my-secret
authSecret: devworkspace-backup-registry-auth
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR description says this fixed name is a workaround for a bug where the restore init container cannot access the pull secret. Is this the permanent intended behavior, or should the docs note this as a known limitation that may change?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's right now a known limitation, which we will likely change for the next release

dkwon17 and others added 3 commits March 27, 2026 11:01
…-oci-registry.adoc

Co-authored-by: Gaurav Trivedi <90042568+gtrivedi88@users.noreply.github.com>
…-oci-registry.adoc

Co-authored-by: Gaurav Trivedi <90042568+gtrivedi88@users.noreply.github.com>
Signed-off-by: David Kwon <dakwon@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants